Skip to content

Clean up plugin binding store boundaries#572

Closed
RhysSullivan wants to merge 1 commit intocodex/effect-lint-baselinefrom
codex/effect-lint-binding-stores
Closed

Clean up plugin binding store boundaries#572
RhysSullivan wants to merge 1 commit intocodex/effect-lint-baselinefrom
codex/effect-lint-binding-stores

Conversation

@RhysSullivan
Copy link
Copy Markdown
Owner

@RhysSullivan RhysSullivan commented May 6, 2026

Summary

  • decode binding-store JSON columns with Effect Schema instead of JSON.parse
  • add local row projection schemas for MCP and Google Discovery binding stores
  • remove redundant primitive casts around source, binding, and credential rows

Verification

  • bunx oxlint -c .oxlintrc.jsonc packages/plugins/google-discovery/src/sdk/binding-store.ts packages/plugins/mcp/src/sdk/binding-store.ts --format json
  • bun run typecheck (packages/plugins/google-discovery)
  • bun run typecheck (packages/plugins/mcp)
  • bunx vitest run src/sdk/plugin.test.ts (packages/plugins/google-discovery)
  • bunx vitest run src/sdk/plugin.test.ts (packages/plugins/mcp)

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented May 6, 2026

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs		 executor-marketing 5cdd5c6 Commit Preview URL

Branch Preview URL		 May 06 2026, 01:47 AM

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented May 6, 2026

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs		 executor-cloud 5cdd5c6 May 06 2026, 01:47 AM

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented May 6, 2026

Open in StackBlitz

@executor-js/cli

npm i https://pkg.pr.new/@executor-js/cli@572

@executor-js/config

npm i https://pkg.pr.new/@executor-js/config@572

@executor-js/execution

npm i https://pkg.pr.new/@executor-js/execution@572

@executor-js/sdk

npm i https://pkg.pr.new/@executor-js/sdk@572

@executor-js/storage-core

npm i https://pkg.pr.new/@executor-js/storage-core@572

@executor-js/codemode-core

npm i https://pkg.pr.new/@executor-js/codemode-core@572

@executor-js/runtime-quickjs

npm i https://pkg.pr.new/@executor-js/runtime-quickjs@572

@executor-js/plugin-file-secrets

npm i https://pkg.pr.new/@executor-js/plugin-file-secrets@572

@executor-js/plugin-google-discovery

npm i https://pkg.pr.new/@executor-js/plugin-google-discovery@572

@executor-js/plugin-graphql

npm i https://pkg.pr.new/@executor-js/plugin-graphql@572

@executor-js/plugin-keychain

npm i https://pkg.pr.new/@executor-js/plugin-keychain@572

@executor-js/plugin-mcp

npm i https://pkg.pr.new/@executor-js/plugin-mcp@572

@executor-js/plugin-onepassword

npm i https://pkg.pr.new/@executor-js/plugin-onepassword@572

@executor-js/plugin-openapi

npm i https://pkg.pr.new/@executor-js/plugin-openapi@572

executor

npm i https://pkg.pr.new/executor@572

commit: 5cdd5c6

@RhysSullivan
Copy link
Copy Markdown
Owner Author

RhysSullivan commented May 6, 2026

Closing as superseded by newer audited cleanup PRs covering the same lint file set: #648, #650, #657.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@RhysSullivan